Skip to content

Report problems when writing files and scripts#3301

Merged
imobachgs merged 3 commits intomasterfrom
agama-files-errors
Mar 19, 2026
Merged

Report problems when writing files and scripts#3301
imobachgs merged 3 commits intomasterfrom
agama-files-errors

Conversation

@imobachgs
Copy link
Copy Markdown
Contributor

@imobachgs imobachgs commented Mar 18, 2026

Problem

When something goes wrong when retrieving/writing files or scripts, Agama just fails. For instance, if you are loading the configuration and a script is not accessible, Agama just stops the SetConfig action. Nothing is reported.

Solution

Notify the user when it happens and offer a way to "retry" the operation.

Testing

  • Tested manually

Screenshots

Failed to parse the URL Failed to parse the URL

Follow-up tasks

  • Translate the error. We need to discuss a good solution.
  • Improve the error presentation.

@imobachgs imobachgs force-pushed the agama-files-errors branch from 2784a4b to f0b74f1 Compare March 18, 2026 23:48
@imobachgs imobachgs marked this pull request as ready for review March 19, 2026 06:56
Copy link
Copy Markdown
Contributor

@joseivanlopez joseivanlopez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@imobachgs imobachgs merged commit 193575d into master Mar 19, 2026
18 of 20 checks passed
@imobachgs imobachgs deleted the agama-files-errors branch March 19, 2026 07:21
@imobachgs imobachgs mentioned this pull request Apr 14, 2026
imobachgs added a commit that referenced this pull request Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants